Skip to content

README: Highlights our conan bintray repository#2158

Merged
mnordsletten merged 3 commits intoincludeos:devfrom
staiyeba:patch-1
May 9, 2019
Merged

README: Highlights our conan bintray repository#2158
mnordsletten merged 3 commits intoincludeos:devfrom
staiyeba:patch-1

Conversation

@staiyeba
Copy link
Copy Markdown
Contributor

@staiyeba staiyeba commented May 3, 2019

No description provided.

MagnusS
MagnusS previously requested changes May 6, 2019
Copy link
Copy Markdown
Member

@MagnusS MagnusS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The > formats the text as a quote + extra space after "at". Is a word missing in the last sentence?

@staiyeba
Copy link
Copy Markdown
Contributor Author

staiyeba commented May 6, 2019

I wanted it to standout, so I put it in a quote. The sentence should look like the image below, so the URL to bintray is visible after "at" on the README as well, like Alf asked.

Screenshot 2019-05-06 at 10 31 03

@mnordsletten mnordsletten requested a review from MagnusS May 8, 2019 07:35
@MagnusS
Copy link
Copy Markdown
Member

MagnusS commented May 9, 2019

It’s a markdown quote and may be rendered differently outside github. I think it’s fine to just add emphasis with bold, just remove the >. Last sentence lgtm

Copy link
Copy Markdown
Contributor

@mnordsletten mnordsletten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All requested changes have been implemented

@mnordsletten mnordsletten dismissed MagnusS’s stale review May 9, 2019 08:02

Changes have been implemented

@mnordsletten mnordsletten merged commit b1e3856 into includeos:dev May 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants